Skip to content

Conversation

@DaveCTurner
Copy link
Contributor

Backports the following commits to 8.3:

Today nodes started in an `InternalTestCluster` use `transport.port: 0`
and `http.port: 0` which selects a port from the ephemeral range. This
range is also used by other tests, notably REST tests, and this can lead
to collisions and consequent failures when nodes restart.

This commit restricts the range of ports using the same algorithm as in
`ESTestCase`, avoiding[^1] such collisions.

[^1]: technically this isn't quite enough because the ephemeral range on
some CI workers overlaps the ranges chosen by `ESTestCase`, but that's a
separate issue tracked in elastic#87734

Closes elastic#87448
@DaveCTurner DaveCTurner added :Delivery/Build Build or test infrastructure >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Delivery Meta label for Delivery team labels Jun 21, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team >test Issues or PRs that are addressing/adding tests v8.3.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants